Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding geospatial datatypes #71

Merged
merged 1 commit into from
Dec 9, 2024
Merged

Adding geospatial datatypes #71

merged 1 commit into from
Dec 9, 2024

Conversation

phazonoverload
Copy link
Contributor

No description provided.

Copy link

linear bot commented Dec 6, 2024

DX-504 Add the missing data types to the docs in the glossary under the "Data Type Superset" section

Describe the Request

While going through the code. I noticed that there are more data types than are listed in the docs. I don't know how they are used but I think they should be listed with all the other data types that are in the docs.

In the directus/contants package there is definition of the data types in the file located here:

  • packages/constants/src/fields.ts

I've included a screenshot:

image

These highlighted data types are not listed in the docs and I think they should be. If someone could provide a good definition for each then we can add them to the glossary here:

https://docs.directus.io/user-guide/overview/glossary.html#data-type-superset

The actual change needs to happen in the docs file located here:

  • docs/user-guide/overview/glossary.md

Maintenance Strategy

I don't think datatypes are being added anymore and if they are it should be a long time before more will be added to the core.

Copy link

netlify bot commented Dec 6, 2024

Deploy Preview for directus-docs-rewrite ready!

Name Link
🔨 Latest commit fbc70a8
🔍 Latest deploy log https://app.netlify.com/sites/directus-docs-rewrite/deploys/6752b297d17be90008f9e523
😎 Deploy Preview https://deploy-preview-71--directus-docs-rewrite.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Dec 6, 2024

Deploy Preview for directus-docs-v2 ready!

Name Link
🔨 Latest commit fbc70a8
🔍 Latest deploy log https://app.netlify.com/sites/directus-docs-v2/deploys/6752b297d9a15c00085d33d4
😎 Deploy Preview https://deploy-preview-71--directus-docs-v2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@hola-soy-milk hola-soy-milk merged commit 3a02109 into main Dec 9, 2024
10 of 11 checks passed
@hola-soy-milk hola-soy-milk deleted the DX-504 branch December 9, 2024 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants